-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update the inserter's block preview to use the AutoHeightPreview #20817
Conversation
Size Change: -37 B (0%) Total Size: 864 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it, works well, thank you, and nice work on so much red! The console error is also gone.
I left one comment, but other than that, ship it.
@@ -229,6 +220,7 @@ $block-inserter-search-height: 38px; | |||
border: $border-width solid $light-gray-500; | |||
border-radius: $radius-round-rectangle; | |||
align-items: center; | |||
min-height: 150px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use a variable here? Or at least something grid friendly like $grid-unit-60 * 3
? I know no-one will really notice this, but I just don't like arbitrarily picked numbers :)
closes #20713
Fixes the trimmed previews by using au automatic computed height for block previews.